settings: Avoid a redundant check
authorMatthias Clasen <mclasen@redhat.com>
Tue, 8 Sep 2015 23:29:32 +0000 (19:29 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Wed, 9 Sep 2015 10:32:45 +0000 (06:32 -0400)
g_value_transform already returns whether it was successful
or not, so no need to call g_value_type_transformable beforehand.

gdk/x11/gdkscreen-x11.c

index 8b2374e1e44361d9b84ff0fe8591e96a0d37e895..44bb69ffbae38e9be2546f54fbdf151c916b58d6 100644 (file)
@@ -1520,7 +1520,7 @@ gdk_x11_screen_get_setting (GdkScreen   *screen,
   if (setting == NULL)
     goto out;
 
-  if (!g_value_type_transformable (G_VALUE_TYPE (setting), G_VALUE_TYPE (value)))
+  if (!g_value_transform (setting, value))
     {
       g_warning ("Cannot transform xsetting %s of type %s to type %s\n",
                 name,
@@ -1529,8 +1529,6 @@ gdk_x11_screen_get_setting (GdkScreen   *screen,
       goto out;
     }
 
-  g_value_transform (setting, value);
-
   return TRUE;
 
  out: